Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow editing of params at any time #30939

Merged
merged 1 commit into from
May 23, 2023

Conversation

stuartwdouglas
Copy link
Member

Currently it is only possible to edit command line parameters in dev mode after the application has exited. This means that if you have long running applications (or accidently hang your app) you can't restart with different parameters.

@quarkus-bot quarkus-bot bot added the area/core label Feb 7, 2023
@quarkus-bot

This comment has been minimized.

@quarkus-bot

This comment has been minimized.

@ebullient
Copy link
Member

@stuartwdouglas .. does this one just need a rebase? Did you have any other reservations about merging it?

@stuartwdouglas
Copy link
Member Author

I have rebased, but looking at the test results it must be somehow screwing with the remote dev tests.

@quarkus-bot

This comment has been minimized.

@quarkus-bot

This comment has been minimized.

Currently it is only possible to edit command line parameters in dev
mode after the application has exited. This means that if you have long
running applications (or accidently hang your app) you can't restart
with different parameters.
@quarkus-bot
Copy link

quarkus-bot bot commented May 23, 2023

✔️ The latest workflow run for the pull request has completed successfully.

It should be safe to merge provided you have a look at the other checks in the summary.

@geoand geoand merged commit 9b2ee9e into quarkusio:main May 23, 2023
@quarkus-bot quarkus-bot bot added this to the 3.2 - main milestone May 23, 2023
@geoand geoand deleted the edit-cli-any-time branch May 23, 2023 06:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants